Switch coding-agent plugins to the Rust daemon - #3
Open
Stephen Belanger (Qard) wants to merge 2 commits into
Open
Switch coding-agent plugins to the Rust daemon#3Stephen Belanger (Qard) wants to merge 2 commits into
Stephen Belanger (Qard) wants to merge 2 commits into
Conversation
Stephen Belanger (Qard)
force-pushed
the
agent/plugin-daemon-cutover
branch
from
July 30, 2026 15:13
f67fddd to
fce3233
Compare
Stephen Belanger (Qard)
force-pushed
the
agent/daemon-tracing-migration
branch
from
July 30, 2026 15:21
709f7bd to
3e019e4
Compare
Replace the legacy Codex TypeScript runtime and Claude shell state machine with fail-open launchers for bt agents hook. Switch the real-agent integration harness from generated direct-hook fixtures to the production plugin packages so the stacked PR validates the actual cutover on every platform. Signed-off-by: Stephen Belanger <stephen.belanger@braintrustdata.com>
Stephen Belanger (Qard)
force-pushed
the
agent/plugin-daemon-cutover
branch
from
July 30, 2026 15:23
fce3233 to
2654cfb
Compare
Signed-off-by: Stephen Belanger <stephen.belanger@braintrustdata.com>
Abhijeet Prasad (AbhiPrasad)
approved these changes
Jul 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dependencies and landing order
Stacked on #1, which introduces the shared Rust daemon while deliberately retaining the existing plugin implementations.
This PR must remain unmerged until the daemon is shipped in a released
btCLI. The intended order is:bt.Summary
This performs the plugin-side cutover to the shared Rust daemon. Codex and Claude Code tracing hooks become small fail-open Unix and Windows launchers that forward agent events to
bt, installing or upgrading the CLI asynchronously when necessary.The legacy Codex TypeScript server, compiled binaries, build scripts, and tests are removed because trace construction moves to Rust. The legacy Claude shell state machine and its test harness are likewise removed. Package builds become content-only, and validation rejects obsolete runtime directories so the two trace models cannot accidentally ship together.
Both plugins use the shared coding-agent tracing settings for enablement, project selection, turn-end flushing, and additional metadata. Authentication and backend resolution remain owned by
bt.The integration harness switches from #1's generated direct-hook fixtures to the simplified production plugin packages in this PR. Their launchers call the released
bt trace hookcommand family on Unix and Windows, so the cross-platform matrix validates the actual plugin cutover rather than a parallel test-only path.Validation
make testcargo test --manifest-path bt-daemon/Cargo.toml --all-features --locked